Skip to content

[PULP-578] Optimize filtering of repo versions by content#7817

Merged
jobselko merged 1 commit into
pulp:mainfrom
jobselko:7799
Jun 30, 2026
Merged

[PULP-578] Optimize filtering of repo versions by content#7817
jobselko merged 1 commit into
pulp:mainfrom
jobselko:7799

Conversation

@jobselko

@jobselko jobselko commented Jun 25, 2026

Copy link
Copy Markdown
Member

closes #7799

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

@jobselko jobselko self-assigned this Jun 25, 2026
mdellweg
mdellweg reviewed Jun 25, 2026
View reviewed changes

@mdellweg mdellweg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work.

Comment thread pulpcore/app/models/repository.py Outdated
Comment thread pulpcore/app/models/repository.py Outdated
@jobselko jobselko force-pushed the 7799 branch 2 times, most recently from 35787ad to fc896b9 Compare June 25, 2026 12:57
@jobselko

jobselko commented Jun 25, 2026

Copy link
Copy Markdown
Member Author

Measured benefit by repeatedly adding/removing the same content unit into a repo (creating 2 RepositoryVersion and 1 RepositoryContent per cycle):

Without patch:
Response time grows with the number of cycles (e.g. ~0.5s at 100, ~2-3s at 700)

With patch:
Response time stays constant at ~0.45s regardless of the number of cycles

@jobselko jobselko force-pushed the 7799 branch 3 times, most recently from 4462e89 to 5b75d44 Compare June 29, 2026 10:50
mdellweg
mdellweg reviewed Jun 29, 2026
View reviewed changes
Comment thread pulpcore/app/models/repository.py Outdated
@jobselko
Optimize filtering of repo versions by content
a633f92 
closes pulp#7799
Assisted By: Claude Opus 4.6
@jobselko jobselko merged commit 1b04034 into pulp:main Jun 30, 2026
13 of 14 checks passed
@jobselko jobselko deleted the 7799 branch June 30, 2026 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dralley dralley dralley approved these changes
@mdellweg mdellweg mdellweg approved these changes

Assignees

@jobselko jobselko

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Use new memoized field in repository content API filtering

3 participants

@jobselko @dralley @mdellweg